From b4063f876f9f2b514a34870d26d530f520e9c906 Mon Sep 17 00:00:00 2001
From: Alex Crichton <alex@alexcrichton.com>
Date: Thu, 31 Jul 2014 12:18:59 -0700
Subject: [PATCH] Be sure to not emit a lone `?` in a URL

---
 src/cargo/core/source.rs | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/cargo/core/source.rs b/src/cargo/core/source.rs
index f60c076cb..9091a0ef9 100644
--- a/src/cargo/core/source.rs
+++ b/src/cargo/core/source.rs
@@ -247,15 +247,12 @@ impl SourceId {
                 let mut url = url.to_url().unwrap();
                 let mut reference = "master".to_string();
                 let pairs = url.query_pairs().unwrap_or(Vec::new());
-                url.set_query_from_pairs(pairs.iter().filter(|&&(ref k, ref v)| {
+                for &(ref k, ref v) in pairs.iter() {
                     if k.as_slice() == "ref" {
                         reference = v.clone();
-                        false
-                    } else {
-                        true
                     }
-                }).map(|&(ref a, ref b)| (a.as_slice(), b.as_slice())));
-
+                }
+                url.query = None;
                 let precise = mem::replace(&mut url.fragment, None);
                 SourceId::for_git(&url, reference.as_slice(), precise)
             },
-- 
2.30.2